Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

feat(analytics): Refund status serialization issue for ckh analytics #5199

Merged
merged 1 commit into from
Jul 4, 2024

Conversation

Abhitator216
Copy link
Contributor

@Abhitator216 Abhitator216 commented Jul 4, 2024

Type of Change

  • Bugfix
  • New feature
  • Enhancement
  • Refactoring
  • Dependency updates
  • Documentation
  • CI/CD

Description

RefundStatus Enum does not support snake_case, so serde directive were added.

Additional Changes

  • This PR modifies the API contract
  • This PR modifies the database schema
  • This PR modifies application configuration/environment variables

Motivation and Context

How did you test it?

Run a Query :

{topic="hyperswitch-refund-events", source!="vector"} |= `ref_XPgVve1i7ISdDXfnxNy0` | json | line_format `{{.refund_status}}`

the Output should be :

2024-07-04 15:37:41.486 failure
2024-07-04 15:37:41.486 pending
2024-07-04 15:37:40.998 pending

Checklist

  • I formatted the code cargo +nightly fmt --all
  • I addressed lints thrown by cargo clippy
  • I reviewed the submitted code
  • I added unit tests for my changes where possible

@Abhitator216 Abhitator216 requested a review from a team as a code owner July 4, 2024 08:35
@Abhitator216 Abhitator216 linked an issue Jul 4, 2024 that may be closed by this pull request
@Abhitator216 Abhitator216 changed the title fixed refund status analytics bug [Analytics] - Refund status serialization issue for ckh analytics Jul 4, 2024
@Abhitator216 Abhitator216 changed the title [Analytics] - Refund status serialization issue for ckh analytics feat(analytics) : Refund status serialization issue for ckh analytics Jul 4, 2024
@Abhitator216 Abhitator216 changed the title feat(analytics) : Refund status serialization issue for ckh analytics feat(analytics): Refund status serialization issue for ckh analytics Jul 4, 2024
@Abhitator216 Abhitator216 self-assigned this Jul 4, 2024
@likhinbopanna likhinbopanna added this pull request to the merge queue Jul 4, 2024
Merged via the queue into main with commit 755d968 Jul 4, 2024
27 of 31 checks passed
@likhinbopanna likhinbopanna deleted the fix/refund-status-serialization-ckh-analytics branch July 4, 2024 12:02
pixincreate added a commit that referenced this pull request Jul 5, 2024
…ify-cypress

* 'main' of github.com:juspay/hyperswitch: (22 commits)
  refactor: Adding millisecond to Kafka timestamp (#5202)
  chore(version): 2024.07.05.0
  fix(user_auth_method): make id option in auth select (#5213)
  Docs: Updated API - ref for payments (#5172)
  feat(core): add merchant order reference id (#5197)
  feat(analytics): Refund status serialization issue for ckh analytics (#5199)
  fix(router): `override setup_future_usage` filed to on_session based on merchant config (#5195)
  feat(cypress): make tests forcefully skippable (#5176)
  feat(core): Added integrity framework for Authorize and Sync flow with connector as Stripe (#5109)
  ci(cypress): Update card number for adyen and status for paypal (#5192)
  refactor(cypress): error handling and add sync refunds in places where missing (#5128)
  feat(analytics): FRM Analytics (#4880)
  chore(version): 2024.07.04.0
  feat(pm_auth): Added balance check for PM auth bank account (#5054)
  refactor(payment_methods): add appropriate missing logs (#5190)
  refactor(migrations): add commands to make file to run migrations for api v2 (#5169)
  chore(version): 2024.07.03.0
  fix(event): Updated the ApiEventMetric (#5126)
  feat(router): add refunds manual-update api (#5094)
  refactor(payment_link): logs payment links logs coverage (#4918)
  ...
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Analytics] - Refund status serialization issue for ckh analytics
4 participants